Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cors 2075 new saga le sa #394

Merged
merged 34 commits into from
Jan 3, 2024
Merged

Cors 2075 new saga le sa #394

merged 34 commits into from
Jan 3, 2024

Conversation

mina-ats
Copy link
Contributor

@mina-ats mina-ats commented Dec 20, 2023

LegalEntityV2, ServiceAgreementV2, LegalEntitySagaV2, and ServiceAgreementSaga are the new models and sagas for the new bootstrap project. We didn't touch the old ones used by other teams.

Checklist

  • I made sure, I read CONTRIBUTING.md to put right branch prefix as per my need.
  • I made sure to update CHANGELOG.md.
  • I made sure to update Stream Wiki(only valid in case of new stream module or architecture changes).
  • My changes are adequately tested.
  • I made sure all the SonarCloud Quality Gate are passed.

minasharifi and others added 29 commits November 21, 2023 17:26
…ap to old version, initial implementation of ServiceAgreementSagaV2
…ap to old version, initial implementation of ServiceAgreementSagaV2
…ap to old version, initial implementation of ServiceAgreementSagaV2
…ORS-2075-new-saga-LE-SA

# Conflicts:
#	stream-legal-entity/legal-entity-core/src/main/java/com/backbase/stream/ServiceAgreementSagaV2.java
@mina-ats mina-ats requested a review from a team as a code owner December 20, 2023 15:35
Copy link
Collaborator

@uesleilima uesleilima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest for this to become part of the upcoming Stream 4.0 under development on the develop branch

@HighTML
Copy link

HighTML commented Dec 21, 2023

I would suggest for this to become part of the upcoming Stream 4.0 under development on the develop branch

@uesleilima this is an addition, not a breaking change. The new Bootstrap Engine is based on Stream 3, moving to Stream 4 would be a way too big change.

CHANGELOG.md Outdated
@@ -1,6 +1,10 @@
# Changelog
All notable changes to this project will be documented in this file.

## [3.71.0](https://github.com/Backbase/stream-services/compare/3.70.0...3.71.0)
### Added
- Added new models for LE and SA to cover requirements in the new bootstrap. In the new structure, LE supports just master service agreement and custom service agreement has been dropped. Product groups, limits, job profile users, reference job roles, contacts were moved from LE to new SA model. In the new bootstrap, LE and SA are now separated and can be defined in the different JSON files and independently ingested.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good! Just a reminder that limits can be ingested at different levels, you can have SA limits, LE limits and User limits, so removing it from the LE model will create limitations on the saga.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@uesleilima you're right, just removed the limits in the changelog. we have limits for both new SA and LE

@uesleilima uesleilima mentioned this pull request Jan 2, 2024
5 tasks
Copy link

sonarcloud bot commented Jan 3, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

60.7% Coverage on New Code (required ≥ 80%)
12.4% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@uesleilima uesleilima merged commit 839e992 into master Jan 3, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

8 participants